Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

search parameters - update requirements #411

Merged
merged 37 commits into from
Jun 24, 2024
Merged

search parameters - update requirements #411

merged 37 commits into from
Jun 24, 2024

Conversation

f-peverali
Copy link
Contributor

@f-peverali f-peverali commented May 31, 2024 •
edited
Loading

Contributor Pull Request

Description

Solves ANFISK 264

Motivation and Context

How has this been tested?

  • review wird benötig besonders
    • Prüfung der Anpassung in Hinblick auf Kompromissvorschlag (immer noch zu hart oder ggf. zu offen)
    • Konsistenzprüfung zum Rest der Search-Requirements

Snippets or Screenshots (if necessary):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (reduction of requirements)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this IG / specification.
  • My change requires a change to the documentation or narrative (intend) of the IG.
  • I have already updated the documentation / narrative (intend) accordingly.
  • ReleaseNotes for feature/fix are updated in releaseNotes.md

@f-peverali f-peverali marked this pull request as draft June 7, 2024 11:18
@alexzautke alexzautke changed the title update requirements update requirements for search parameters Jun 14, 2024
@f-peverali f-peverali changed the title update requirements for search parameters search parameters - update requirements Jun 14, 2024
@f-peverali f-peverali marked this pull request as draft June 24, 2024 09:10
@f-peverali f-peverali closed this Jun 24, 2024
@f-peverali f-peverali reopened this Jun 24, 2024
@f-peverali f-peverali marked this pull request as ready for review June 24, 2024 09:38
@f-peverali f-peverali merged commit af32744 into rc/main-stufe-4 Jun 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexey-tschudnowsky alexey-tschudnowsky alexey-tschudnowsky left review comments

@alexzautke alexzautke alexzautke approved these changes

@simoneOnFhir simoneOnFhir Awaiting requested review from simoneOnFhir

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@f-peverali @alexzautke @alexey-tschudnowsky @simoneOnFhir